Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(#10609): remove isChromatic on real build #10613

Merged
merged 4 commits into from
Apr 13, 2023
Merged

fix(#10609): remove isChromatic on real build #10613

merged 4 commits into from
Apr 13, 2023

Conversation

acid-chicken
Copy link
Member

What

isChromatic() をプロダクションビルドで消す

Why

Fix #10609

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Apr 13, 2023
@github-actions github-actions bot requested a review from rinsuki April 13, 2023 08:59
@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Merging #10613 (a60421b) into develop (dffefda) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop   #10613   +/-   ##
========================================
  Coverage    78.33%   78.33%           
========================================
  Files          164      164           
  Lines        20445    20446    +1     
  Branches       363      363           
========================================
+ Hits         16016    16017    +1     
  Misses        4429     4429           
Impacted Files Coverage Δ
packages/frontend/src/components/global/MkTime.vue 87.87% <100.00%> (+0.18%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tamaina
Copy link
Contributor

tamaina commented Apr 13, 2023

(プロダクションビルドできるか確認してなかったな?)

Storybookちゃんと動くか確認お願いします

@acid-chicken
Copy link
Member Author

Storybook Publish — 81 stories published. Details

image

なのでビルド通ってる

@tamaina tamaina merged commit 47c7b4b into develop Apr 13, 2023
@tamaina tamaina deleted the isChromatic branch April 13, 2023 14:23
@tamaina
Copy link
Contributor

tamaina commented Apr 13, 2023

👍

na2na-p pushed a commit to na2na-p/misskey that referenced this pull request May 10, 2023
…#10613)

* perf: remove isChromatic on real build

* revert: Revert misskey-dev#10475 in MkTime

This reverts commit 7d11cf8.

* @rollup/plugin-replace as dependencies

* fix pnpm-lock,yaml

---------

Co-authored-by: tamaina <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
2 participants